Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update display.php #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update display.php #39

wants to merge 1 commit into from

Conversation

orwah
Copy link
Contributor

@orwah orwah commented Oct 19, 2016

i guess
this 2 lines must be commented by default
for viewing Buttons images instead of using css button style (as default)

i guess 
this 2 lines must be commented  by default 
for viewing Buttons images instead of using css button style (as default)
@mambax7
Copy link
Contributor

mambax7 commented Oct 19, 2016

CSS buttons are a safer choice because they will work as soon as the text is translated into local language. Images, on the other hand, would have to be created, which takes more time and higher level of skills.

@mambax7
Copy link
Contributor

mambax7 commented Nov 1, 2016

What do you think?

@orwah
Copy link
Contributor Author

orwah commented Nov 1, 2016

it's true .

the only problem is that Users will take along time to specify the files for changing this back to Images. and for adjust the Buttons properties in css.

I think, it would be good to have this as an option in Preferences .

mambax7 added a commit to mambax7/newbb that referenced this pull request Feb 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants